Skip to content

ci(qodana): .editorconfig-Workaround entfernen#62

Merged
tomtastisch merged 1 commit intomainfrom
codex/ci/qodana-editorconfig-workaround-entfernen
Feb 14, 2026
Merged

ci(qodana): .editorconfig-Workaround entfernen#62
tomtastisch merged 1 commit intomainfrom
codex/ci/qodana-editorconfig-workaround-entfernen

Conversation

@tomtastisch
Copy link
Owner

@tomtastisch tomtastisch commented Feb 14, 2026

Ziel & Scope

Fixes #59: Qodana soll mit der echten Repo-Konfiguration laufen. Der .editorconfig-Disable/Restore-Workaround in .github/workflows/qodana.yml wird entfernt.

Non-Goals:

  • Keine inhaltlichen Regel-Aenderungen in .editorconfig (nur Workflow-Hack entfernen).
  • Keine Aenderungen an SECURITY.md.

Umgesetzte Aufgaben (abhaken)

  • Disable/Restore-Schritte fuer .editorconfig aus .github/workflows/qodana.yml entfernt.
  • Qodana laeuft damit gegen die reale Repo-Konfiguration (kein Datei-Rename).
  • Lokale CI-Skripte ausgefuehrt (preflight/build/tests) fuer Regressionschutz.

Nachbesserungen aus Review (iterativ)

  • Falls Qodana mit .editorconfig wieder fehlschlaegt: Root-Cause im Qodana-Log identifizieren und zielgerichtet beheben (konfig statt rename).
  • Optional: use-caches: true separat evaluieren (nur wenn stabil, nicht in diesem PR).

Security- und Merge-Gates

  • Required Checks: alle required Checks sind gruen.
  • Code scanning: security/code-scanning/tools zeigt 0 offene Alerts.
  • Keine offenen Review-Threads (required conversation resolution).

Evidence (auditierbar)

Lokal:

  • bash tools/ci/bin/run.sh preflight -> /Users/tomwerner/RiderProjects/FileClassifier/artifacts/ci/preflight/summary.md
  • bash tools/ci/bin/run.sh build -> /Users/tomwerner/RiderProjects/FileClassifier/artifacts/ci/build/summary.md
  • bash tools/ci/bin/run.sh tests-bdd-coverage -> /Users/tomwerner/RiderProjects/FileClassifier/artifacts/ci/tests-bdd-coverage/summary.md

CI (PR #62):

  • Qodana Workflow Run: 22021300257
  • CI Workflow Run (inkl. preflight/pr-governance): 22021300238

DoD (mindestens 2 pro Punkt)

  • Punkt: Qodana ohne .editorconfig-Workaround
  • DoD 1: Workflow enthaelt keinen Datei-Hack mehr (auditierbar in .github/workflows/qodana.yml).
  • DoD 2: Lokale Build+Tests sind gruen (auditierbar in artifacts/ci/build/summary.md und artifacts/ci/tests-bdd-coverage/summary.md).

Copilot AI review requested due to automatic review settings February 14, 2026 17:20
@github-actions github-actions bot added area:pipeline ci CI/workflow change impl:config versioning:patch Fix/Refactor/Docs/CI/Tooling; requires PATCH bump labels Feb 14, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the temporary .editorconfig workaround from the Qodana workflow, addressing issue #59. The workaround previously disabled .editorconfig before running Qodana and restored it afterward, creating configuration drift. Now Qodana runs with the actual repository configuration.

Changes:

  • Removed temporary .editorconfig disable/restore steps from the Qodana workflow
  • Qodana now runs directly against the repository's actual configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tomtastisch tomtastisch merged commit a6da514 into main Feb 14, 2026
32 checks passed
@tomtastisch tomtastisch deleted the codex/ci/qodana-editorconfig-workaround-entfernen branch February 14, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:pipeline ci CI/workflow change impl:config versioning:patch Fix/Refactor/Docs/CI/Tooling; requires PATCH bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

QDNET/Qodana: .editorconfig workaround entfernen (Best Practice)

1 participant